Skip to content

Update Elasticsearch to 9.2.3 and remove built-in plugins#11

Merged
rophy merged 4 commits intorophy:mainfrom
JeWcLee:9.2.3
Jan 15, 2026
Merged

Update Elasticsearch to 9.2.3 and remove built-in plugins#11
rophy merged 4 commits intorophy:mainfrom
JeWcLee:9.2.3

Conversation

@JeWcLee
Copy link
Contributor

@JeWcLee JeWcLee commented Jan 8, 2026 •
edited by coderabbitai bot
Loading

Following plugins are built in after 8.x, so removed these plugins from Dockerfile

  • ingest-attachment
  • repository-s3

Summary by CodeRabbit

  • Chores
    • Upgraded Elasticsearch base to 9.2.3 (from 8.18.2).
    • Adjusted plugin installation list to match the new Elasticsearch version.
    • Retained key language analysis plugins for multilingual support.
    • Cleaned up the recorded version entry to reflect the upgrade.

✏️ Tip: You can customize this high-level summary in your review settings.

@JeWcLee JeWcLee marked this pull request as ready for review January 13, 2026 10:02
@coderabbitai
Copy link

coderabbitai bot commented Jan 13, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

Dockerfile updated to use elasticsearch:9.2.3; plugin install list changed (removed ingest-attachment and repository-s3) and analysis-ik plugin URL aligned to 9.2.3. elasticsearch-a1/version.txt had its single version line removed (file now empty).

Changes

Cohort / File(s) Summary
Elasticsearch Base Image & Plugins
elasticsearch-a1/Dockerfile		 Base image bumped to elasticsearch:9.2.3; removed ingest-attachment and repository-s3 from the batch install; updated analysis-ik plugin URL to match 9.2.3; retained analysis-icu and analysis-kuromoji.
Version Metadata
elasticsearch-a1/version.txt		 Removed the 8.18.2-20260106-r1 entry; file is now empty (no version string present).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐇 A hop for the base, a twitch for the plug,
I nudged the version, gave old lines a shrug.
Plugins trimmed tidy, URLs set anew,
The rabbit approves — a small change, true! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main changes in the pull request: updating Elasticsearch to 9.2.3 and removing built-in plugins.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1689de6 and 37bd3a9.

📒 Files selected for processing (1)
  • elasticsearch-a1/version.txt
💤 Files with no reviewable changes (1)
  • elasticsearch-a1/version.txt

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@JeWcLee JeWcLee changed the title Elasticsearch add plugins for 9.2.3 Update Elasticsearch to 9.2.3 and remove built-in plugins Jan 14, 2026
rophy
rophy requested changes Jan 15, 2026
View reviewed changes
@rophy rophy merged commit 9be34dd into rophy:main Jan 15, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rophy rophy rophy requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JeWcLee @rophy